Skip to content

Conversation

@dlmarion
Copy link
Contributor

@dlmarion dlmarion commented Jul 3, 2024

This commit modifies the Terraform configuration such that instead of deploying a Telegraf/InfluxDB/Grafana container on the Manager node, it now deploys Telegraf on each node and uses a Timely/Grafana container on the Manager node. The processes running on each node will send metrics to the Telegraf process running locally,which will then forward the metrics to Timely to be inserted into Accumulo. Thus, as the cluster grows larger, more and more metrics will be created and inserted into Accumulo.

@dlmarion dlmarion self-assigned this Jul 3, 2024
@DomGarguilo
Copy link
Member

With these changes, does the Deployment Overview section in the README need to be updated?

https://github.com/apache/accumulo-testing/blob/f191f7b23ebcf28388a33efcf71ed8437f1c8cbc/contrib/terraform-testing-infrastructure/README.md#deployment-overview

@dlmarion
Copy link
Contributor Author

dlmarion commented Oct 7, 2024

With these changes, does the Deployment Overview section in the README need to be updated?

https://github.com/apache/accumulo-testing/blob/f191f7b23ebcf28388a33efcf71ed8437f1c8cbc/contrib/terraform-testing-infrastructure/README.md#deployment-overview

Done in 3bf6d7f

@ctubbsii ctubbsii deleted the branch apache:main December 10, 2024 20:23
@ctubbsii ctubbsii closed this Dec 10, 2024
@ctubbsii ctubbsii reopened this Dec 10, 2024
@ctubbsii ctubbsii changed the base branch from elasticity to main December 10, 2024 20:26
@ctubbsii
Copy link
Member

@dlmarion what's the status of this?

@dlmarion
Copy link
Contributor Author

dlmarion commented Apr 8, 2025

@ctubbsii - status is I was just waiting for review to happen. I'll make sure that it's up date.

Copy link
Member

@ctubbsii ctubbsii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not opposed to merging this, but I'm not clear on its value either. I don't know how much value InfluxDB offered, vs. Timely.

"It's fine, I guess", is about all I can say, if people find it useful for testing Accumulo. I don't frequently use the Terraform stuff, though, and haven't tested this myself.

@dlmarion
Copy link
Contributor Author

dlmarion commented Apr 9, 2025

I'm not opposed to merging this, but I'm not clear on its value either.

The value here is that using Timely instead of InfluxDB places load on Accumulo as Timely stores its data in Accumulo. Having the cluster running inserts data without having to run CI.

@dlmarion dlmarion merged commit a69f2c4 into apache:main Jan 9, 2026
1 check passed
@dlmarion dlmarion deleted the ctg-stack branch January 9, 2026 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants